[RELEASE] cuvs v25.10#1403
Merged
Merged
Conversation
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
This PR removes the OS suffix from devcontainers, allowing the upstream devcontainer images to determine the OS version. Contributes to rapidsai/build-planning#200. Authors: - Bradley Dice (https://github.com/bdice) Approvers: - Kyle Edwards (https://github.com/KyleFromNVIDIA) URL: #1169
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
conda-forge is migrating to gcc 14, so this PR is updating for alignment. See rapidsai/build-planning#188 Authors: - Vyas Ramasubramani (https://github.com/vyasr) Approvers: - Jake Awe (https://github.com/AyodeAwe) - Ben Frederickson (https://github.com/benfred) URL: #1030
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
rapids_config will use `RAPIDS_BRANCH` contents to determine what branch to use Authors: - Robert Maynard (https://github.com/robertmaynard) Approvers: - Bradley Dice (https://github.com/bdice) - Ben Frederickson (https://github.com/benfred) URL: #1143
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
Forward-merge branch-25.08 into branch-25.10
This PR introduce a custom js script to regroup overloads into a single link. Another alternative would be to have the brief of every function in there instead of grouping them, so that it could look like ``` - fit(float, int) - fit(double, int) - fit(float, int64) - ... ``` Before: <img width="273" height="832" alt="grafik" src="https://github.com/user-attachments/assets/e088e597-e442-4338-b2ee-1bd26e4300a6" /> After: <img width="296" height="295" alt="grafik" src="https://github.com/user-attachments/assets/63279466-1245-43fe-b873-297edb697128" /> Authors: - Micka (https://github.com/lowener) Approvers: - Ben Frederickson (https://github.com/benfred) URL: #1377
…and int32) (#1283) With the introduction of `CuVSMatrix` as the type to provide an input dataset, we expanded the types that can be used in indexing. With this PR we aim to do the same query/search: - Use a `CuVSMatrix` for `CagraQuery` too - Use the correct types for sizes in `CagraIndex#search` There is more we can do there, e.g. use `CuVSDeviceMatrix` for storing the results so we don't have to explicitly copy them (all duplicated code). That would be follow-up work. Authors: - Lorenzo Dematté (https://github.com/ldematte) - Ben Frederickson (https://github.com/benfred) Approvers: - Ben Frederickson (https://github.com/benfred) - Chris Hegarty (https://github.com/ChrisHegarty) URL: #1283
Authors: - Kyle Edwards (https://github.com/KyleFromNVIDIA) Approvers: - Robert Maynard (https://github.com/robertmaynard) URL: #1385
Fixes #853 Authors: - Tarang Jain (https://github.com/tarang-jain) Approvers: - Ben Frederickson (https://github.com/benfred) - Ben Karsin (https://github.com/bkarsin) URL: #1112
This PR adds full support for row strides to `CuVSMatrix`.
There was already some support for them in `CuVSDeviceMatrix`, as we needed to at least to save them to handle matrices built from tensors (e.g. resulting from a cuvs computation) and/or to create tensors for that kind of data (again, to pass them to the C API).
With this PR we introduce ways to:
- allocate host and device matrices with row stride != from the number of columns (e.g. padding/additional info at the end of row vectors)
- map a pre-allocated memory region to a `CuVSMatrix` taking row stride into account
- copy between different matrices with different strides
- populate data taking the target matrix stride into account
- read data from matrices (e.g. read rows or copy to heap arrays) taking the source matrix stride into account
The API changes include also the introduction of column strides ("gaps" between elements withing the same vector); however this is not implemented yet. To the best of my knowledge, cuvs functions do not support these _yet_. I wanted to introduce them as we need them in some scenarios (e.g. round-tripping from `DLTensor`s), and I did not want to change the API once again if/when we are going to support them.
However, I am open to drop `columnStride` if we do not want it. Let me know.
Authors:
- Lorenzo Dematté (https://github.com/ldematte)
- Ben Frederickson (https://github.com/benfred)
- MithunR (https://github.com/mythrocks)
Approvers:
- Chris Hegarty (https://github.com/ChrisHegarty)
- MithunR (https://github.com/mythrocks)
URL: #1345
#1350) This PR : - updates doc build to treat warnings as errors - fixes an issue in the MG ANNs Python API doc - fixes several doc issues that resulted in warnings Closes #1382 Authors: - Victor Lafargue (https://github.com/viclafargue) Approvers: - Ben Frederickson (https://github.com/benfred) - James Lamb (https://github.com/jameslamb) - Corey J. Nolet (https://github.com/cjnolet) URL: #1350
Authors: - Tarang Jain (https://github.com/tarang-jain) Approvers: - Ben Frederickson (https://github.com/benfred) URL: #1399
This PR patches FAISS for some missing Thrust includes. This is needed to support building with CCCL 3.1.x. RAPIDS 25.10 will still use CCCL 3.0.x but we want to make it possible to build with 3.1.x in 25.10. xref: #1329 (This backports a fix from that testing PR, which is targeting 25.12) I am upstreaming the patch here: facebookresearch/faiss#4597 Authors: - Bradley Dice (https://github.com/bdice) - Corey J. Nolet (https://github.com/cjnolet) Approvers: - Tarang Jain (https://github.com/tarang-jain) - Divye Gala (https://github.com/divyegala) URL: #1398
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❄️ Code freeze for
branch-25.10and v25.10 releaseWhat does this mean?
Only critical/hotfix level issues should be merged into
branch-25.10until release (merging of this PR).What is the purpose of this PR?
branch-25.10intomainfor the release